Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor OC\Server::getEncryptionKeyStorage #40133

Conversation

summersab
Copy link
Contributor

@summersab summersab commented Aug 30, 2023

This PR refactors the deprecated method OC\Server::getEncryptionKeyStorage and replaces it with OC\Server::get(\OCP\Encryption\Keys\IStorage::class) throughout the entire NC codebase (excluding ./apps and ./3rdparty).

Additionally, where necessary, the OCP\Encryption\Keys\IStorage class is imported via the use directive.

@summersab summersab force-pushed the refactor/OC-Server-getEncryptionKeyStorage branch 2 times, most recently from 1bd2464 to d2d8669 Compare August 30, 2023 02:29
@solracsf solracsf added the 3. to review Waiting for reviews label Aug 30, 2023
@solracsf solracsf added this to the Nextcloud 28 milestone Aug 30, 2023
@skjnldsv skjnldsv mentioned this pull request Nov 1, 2023
This was referenced Nov 6, 2023
This was referenced Nov 14, 2023
@blizzz blizzz modified the milestones: Nextcloud 28, Nextcloud 29 Nov 23, 2023
This was referenced Mar 12, 2024
@skjnldsv skjnldsv requested review from a team, ArtificialOwl, icewind1991 and Fenn-CS and removed request for a team March 15, 2024 12:11
Signed-off-by: Andrew Summers <18727110+summersab@users.noreply.github.com>
@skjnldsv skjnldsv force-pushed the refactor/OC-Server-getEncryptionKeyStorage branch from d2d8669 to 6cc147c Compare March 15, 2024 12:11
lib/private/Encryption/EncryptionWrapper.php Outdated Show resolved Hide resolved
lib/private/Encryption/EncryptionWrapper.php Outdated Show resolved Hide resolved
@Altahrim Altahrim removed the 3. to review Waiting for reviews label Mar 15, 2024
@skjnldsv skjnldsv added the 2. developing Work in progress label Mar 15, 2024
This was referenced Mar 18, 2024
@skjnldsv skjnldsv mentioned this pull request Mar 28, 2024
81 tasks
@skjnldsv skjnldsv modified the milestones: Nextcloud 29, Nextcloud 30 Mar 28, 2024
Co-authored-by: Benjamin Gaussorgues <github-fa3ie@altahrim.net>
Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
@skjnldsv skjnldsv requested a review from Altahrim May 30, 2024 12:42
@skjnldsv skjnldsv added 3. to review Waiting for reviews and removed 2. developing Work in progress labels May 30, 2024
@skjnldsv skjnldsv merged commit 57cd95f into nextcloud:master Jun 4, 2024
145 of 154 checks passed
@blizzz blizzz mentioned this pull request Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants